Skip to content

Take-Home Assignment: Tests, Bug Fix, and Assign Task Feature#44

Open
kKrishGupta wants to merge 5 commits into
rohit-ups:mainfrom
kKrishGupta:main
Open

Take-Home Assignment: Tests, Bug Fix, and Assign Task Feature#44
kKrishGupta wants to merge 5 commits into
rohit-ups:mainfrom
kKrishGupta:main

Conversation

@kKrishGupta
Copy link
Copy Markdown

Overview

This PR completes the take-home assignment by adding comprehensive tests, identifying and fixing bugs, and implementing a new feature.


What was done

1. Testing

  • Added unit tests for service layer
  • Added integration tests for API routes
  • Covered both happy paths and edge cases
  • Achieved ~85%+ test coverage

2. Bugs Identified

The following issues were discovered through testing:

  • Incorrect pagination logic (page offset calculation)
  • Status filtering allowed partial matches
  • Task completion reset priority unexpectedly

Details are documented in BUG_REPORT.md.


3. Bug Fix

Fixed pagination logic:

const offset = (page - 1) * limit;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant